Skip to content

feat: always allow command#120

Merged
Dimillian merged 8 commits intoDimillian:mainfrom
baptisteArno:approve-and-add-to-rules
Jan 19, 2026
Merged

feat: always allow command#120
Dimillian merged 8 commits intoDimillian:mainfrom
baptisteArno:approve-and-add-to-rules

Conversation

@baptisteArno
Copy link
Contributor

@baptisteArno baptisteArno commented Jan 19, 2026

CleanShot 2026-01-19 at 12 10 16@2x

  • add "Always allow" approval action that writes Codex rules
  • remember approvals in-session to avoid re-prompts
  • skip duplicate default.rules writes

@baptisteArno baptisteArno changed the title Remember command approvals feat: always allow command Jan 19, 2026
@Dimillian
Copy link
Owner

@codex review

@Dimillian
Copy link
Owner

Seems like it also have your themes changes?

@baptisteArno
Copy link
Contributor Author

I actually prefer without transparency so I have the setting on haha

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a65b39b8e0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@baptisteArno baptisteArno force-pushed the approve-and-add-to-rules branch from a65b39b to 4eae796 Compare January 19, 2026 12:57
@Dimillian
Copy link
Owner

@codex review

@Dimillian
Copy link
Owner

Some typecheck error

Error: src/App.tsx(1198,5): error TS2353: Object literal may only specify known properties, and 'themePreference' does not exist in type 'LayoutNodesOptions'.
Error: src/App.tsx(1198,22): error TS2304: Cannot find name 'resolvedTheme'.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7a5e383d47

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@baptisteArno
Copy link
Contributor Author

ready to review again @Dimillian, thanks for your patience 🙏

@Dimillian Dimillian merged commit e0e9dc6 into Dimillian:main Jan 19, 2026
gersmann pushed a commit to gersmann/codex-monitor-web that referenced this pull request Mar 10, 2026
Co-authored-by: Thomas Ricouard <ricouard77@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants